Skip to content

DOC - add markdown version of the docs#2191

Merged
rcap107 merged 8 commits into
skrub-data:mainfrom
rcap107:doc-add-markdown-documentation
Jul 2, 2026
Merged

DOC - add markdown version of the docs#2191
rcap107 merged 8 commits into
skrub-data:mainfrom
rcap107:doc-add-markdown-documentation

Conversation

@rcap107

@rcap107 rcap107 commented Jun 25, 2026

Copy link
Copy Markdown
Member

This PR replaces part of #2173. Specifically, it adds the sphinx addons that generate the markdown documentation and the llms.txt file.

@rcap107

rcap107 commented Jun 25, 2026

Copy link
Copy Markdown
Member Author

The llms.txt file can be accessed from the "rendered docs" by replacing index.html with llms.txt:

https://output.circle-artifacts.com/output/job/6c470cf4-6746-489a-9d82-60da7c6f5e61/artifacts/0/doc/index.html
https://output.circle-artifacts.com/output/job/6c470cf4-6746-489a-9d82-60da7c6f5e61/artifacts/0/doc/llms.txt

Markdown files can be tested by adding the path reported in the llms.txt file after doc.

Note that the paths have a different root: _sources/modules/default_wrangling/table_vectorizer.md (the _sources part)

Comment thread doc/Makefile
SKB_TABLE_REPORT_VERBOSITY=0 $(SPHINXBUILD) -D plot_gallery=0 -b html $(ALLSPHINXOPTS) $(BUILDDIR)/html
SKB_TABLE_REPORT_VERBOSITY=0 SKIP_JUPYTERLITE=1 $(SPHINXBUILD) -D markdown_uri_doc_suffix="html.md" -D plot_gallery=0 -b html $(ALLSPHINXOPTS) $(BUILDDIR)/html
# Build markdown sources so llms.txt links point to .md files
SKB_TABLE_REPORT_VERBOSITY=0 SKIP_JUPYTERLITE=1 $(SPHINXBUILD) -D markdown_uri_doc_suffix="html.md" -D plot_gallery=0 -b markdown $(ALLSPHINXOPTS) $(BUILDDIR)/markdown

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

does this double the time it takes to make html-noplot? (asking for a friend who finds it too slow already) if yes, maybe we can have a html-noplot-nomarkdown or similar

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes it does, I'll add a command to only build the docs

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cool thanks!

@rcap107 rcap107 merged commit b81197e into skrub-data:main Jul 2, 2026
29 checks passed
@rcap107 rcap107 deleted the doc-add-markdown-documentation branch July 2, 2026 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants